Skip to content
This repository was archived by the owner on May 6, 2020. It is now read-only.

fix(middleware): move to 1.10 style middleware#955

Merged
helgi merged 1 commit intodeis:masterfrom
helgi:django_1.10
Aug 22, 2016
Merged

fix(middleware): move to 1.10 style middleware#955
helgi merged 1 commit intodeis:masterfrom
helgi:django_1.10

Conversation

@helgi
Copy link
Copy Markdown
Contributor

@helgi helgi commented Aug 5, 2016

Waiting for zestedesavoir/django-cors-middleware#25 to land as we are blocked on that for now

@helgi helgi added this to the v2.4 milestone Aug 5, 2016
@helgi helgi self-assigned this Aug 5, 2016
@helgi helgi modified the milestones: v2.5, v2.4 Aug 12, 2016
@helgi helgi force-pushed the django_1.10 branch 2 times, most recently from c8252c1 to 7ae31f6 Compare August 14, 2016 06:21
@helgi
Copy link
Copy Markdown
Contributor Author

helgi commented Aug 14, 2016

Made the Certificate fields UTC only, otherwise there are bunch of warnings. It's a bit of an assumption but it's not very time sensitive information.

Also seems the way a property is now dealt with a bit differently so changed that as well

@codecov-io
Copy link
Copy Markdown

codecov-io commented Aug 15, 2016

Current coverage is 86.27% (diff: 90.00%)

Merging #955 into master will increase coverage by 0.11%

@@             master       #955   diff @@
==========================================
  Files            28         28          
  Lines          3159       3163     +4   
  Methods           0          0          
  Messages          0          0          
  Branches        537        537          
==========================================
+ Hits           2722       2729     +7   
+ Misses          290        288     -2   
+ Partials        147        146     -1   

Powered by Codecov. Last update 3f3be4d...b6c4b0b

@helgi helgi merged commit 0d454ee into deis:master Aug 22, 2016
@helgi helgi deleted the django_1.10 branch August 22, 2016 20:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants